Skip to content

feat(ui): add unread-notification dot indicator - #333

Merged
gmmcosta15 merged 4 commits into
devfrom
feature/notification-unread-dot
Sep 29, 2026
Merged

gmmcosta15 merged 4 commits into
devfrom
feature/notification-unread-dot

Conversation

@gmmcosta15

Copy link
Copy Markdown
Collaborator

Description

  • Feature
  • Bug fix
  • Code refactor
  • Documentation

Adds a red-dot "unread" indicator (mirroring the existing update-available dot) to ui.notification_btn and conn_window.notification_button, driven by a new NotificationPage.has_new_notification signal.

  • IconButton: paint a red dot via setShowNotification()/_paint_notification().
  • NotificationPage: emit has_new_notification(True) on new notification, (False) on view/panel clear.
  • MainWindow: wire the signal to both buttons.
  • Add unit tests for the signal behavior.

@gmmcosta15 gmmcosta15 added the enhancement New feature or request. label Sep 9, 2026
@gmmcosta15 gmmcosta15 self-assigned this Sep 9, 2026
@gmmcosta15
gmmcosta15 force-pushed the feature/notification-unread-dot branch from 9965125 to 0d6f365 Compare September 9, 2026 09:58
Comment thread BlocksScreen/lib/panels/widgets/MainWindow/notificationPage.py

@Robert0Mart Robert0Mart left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gmmcosta15
gmmcosta15 added this pull request to stack #335 September 9, 2026 14:35
@gmmcosta15 gmmcosta15 mentioned this pull request Sep 23, 2026
1 task done
@gmmcosta15
gmmcosta15 force-pushed the feature/notification-unread-dot branch from 0b77dc8 to c7c8356 Compare September 24, 2026 09:46
@gmmcosta15
gmmcosta15 merged commit 4db43da into dev Sep 29, 2026
8 checks passed
@gmmcosta15
gmmcosta15 deleted the feature/notification-unread-dot branch September 29, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants